Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

delete_all raise an error if a limit is provided - fixes #4979 #6089

Merged
merged 1 commit into from May 1, 2012
Merged

delete_all raise an error if a limit is provided - fixes #4979 #6089

merged 1 commit into from May 1, 2012

Conversation

frodsan
Copy link
Contributor

@frodsan frodsan commented Apr 30, 2012

No description provided.

@rafaelfranca
Copy link
Member

cc @tenderlove

@carlosantoniodasilva
Copy link
Member

/cc @tenderlove

@ghost ghost assigned tenderlove Apr 30, 2012
tenderlove added a commit that referenced this pull request May 1, 2012
delete_all raise an error if a limit is provided - fixes #4979
@tenderlove tenderlove merged commit 7d2df5f into rails:master May 1, 2012
@frodsan
Copy link
Contributor Author

frodsan commented May 2, 2012

@carlosantoniodasilva this should be backported to 3-2 too? I'm not sure, this was my first commit merged.

@rafaelfranca
Copy link
Member

I think so. Please open a new pull request to 3-2-stable

Rafael Mendonça França
On May 2, 2012 7:00 PM, "Francesco Rodríguez" <
reply@reply.github.com>
wrote:

@carlosantoniodasilva this should be backported to 3-2 too?


Reply to this email directly or view it on GitHub:
#6089 (comment)

@frodsan
Copy link
Contributor Author

frodsan commented May 2, 2012

@rafaelfranca @tenderlove ok, done. thanks!

@frodsan frodsan deleted the delete_all_limit branch April 11, 2013 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants